Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.26%. Comparing base (ab02f04) to head (f26a0a5).

@@            Coverage Diff             @@
##             main     #143      +/-   ##
==========================================
- Coverage   99.26%   99.26%   -0.01%     
==========================================
  Files          27       27              
  Lines         820      816       -4     
==========================================
- Hits          814      810       -4     
  Misses          6        6              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Hey @danielmaier42 maybe you can help me out here 🙂

The newly added integration test for Users::executeActionsEmail() is failing because the user email is missing from the request.

Looking at the Keycloak docs, I can't see any parameter for the email - do you happen to know how the endpoint is supposed to be called?

@fschmtt , the error message should be relating to the user profile. The user must have an email address assigned, otherwise you cannot execute any email actions. The request itself only has an array of actions as body, so that should be working now. 😊

Are you able to add an email address for your user on which you are testing?

But afar from that it hardly depends on how far you want to go with testing. Do you want to test, that the email is actually sent? Then you would need an smtp setup as well and maybe some Mailhog Docker setup. But i guess it should be good enough for now, if you have an email assigned and the request is getting executed with an 204 as Response.

Thanks for clarifying! I had to set the user to enabled and set an email address.

I didn't decide on how far I wanna go with integration testing; but it definitely helps to identify issues if as much as possible is covered with actual integration tests e.g. in case the Keycloak API has changed (which has happened in the past 😅).

Closed in favor of #142